Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add remoteFingerprints method to PeerConnection #1206

Conversation

achingbrain
Copy link
Contributor

Returns a vector that contains the certificate fingerprints used by the connection to the remote peer.

Fingerprints are deduplicated before adding to the vector as the verifyFingerprint method can be invoked multiple times during the connection lifecycle.

Closes #1203
Refs #1166

Returns a vector that contains the certificate fingerprints used
by the connection to the remote peer.

Closes paullouisageneau#1203
Refs paullouisageneau#1166
@achingbrain achingbrain force-pushed the feat/add-remotefingerprints-method branch from be5a4d9 to 9d03db6 Compare June 11, 2024 08:39
src/impl/peerconnection.cpp Outdated Show resolved Hide resolved
src/impl/peerconnection.cpp Outdated Show resolved Hide resolved
include/rtc/peerconnection.hpp Outdated Show resolved Hide resolved
src/impl/peerconnection.cpp Outdated Show resolved Hide resolved
src/impl/peerconnection.hpp Outdated Show resolved Hide resolved
src/impl/peerconnection.cpp Outdated Show resolved Hide resolved
src/impl/peerconnection.cpp Outdated Show resolved Hide resolved
include/rtc/peerconnection.hpp Outdated Show resolved Hide resolved
src/peerconnection.cpp Outdated Show resolved Hide resolved
Copy link
Owner

@paullouisageneau paullouisageneau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good, thank you!

@paullouisageneau paullouisageneau merged commit b756b5a into paullouisageneau:master Jun 14, 2024
12 checks passed
@achingbrain achingbrain deleted the feat/add-remotefingerprints-method branch June 14, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the remote description after SDP negotiation?
2 participants